Require helm v2.16.0 explicitly and minor CI updates #1911
Conversation
consideRatio
changed the title
|
@consideRatio, may I ask why the required Even for z2jh 0.10.6 the docs say the required k8s version is 1.11. |
|
@michzimny the key need to upgrade to a higher k8s version related to the user-scheduler pod. We run a We had also stopped testing against k8s so old as k8s 1.11. I'm not sure if the docs is correct of not, but you can inspect the required version according to the Chart.yaml of the Helm chart. I've lost track of things in detail at this point. |
I made use of a helm template function (list
concat) made available in helm v2.16 but not helm v2.15, due to that we need to explicitly require helm v2.16+. I know this has caused a bug for one user at least who ran into this error when upgrading to 0.10.2. By specifying this explicitly, anyone upgrading will from now on at least get a hard failure directly about this.Besides that, I also add some small changes to the CI, such as strict linting of helm and passing a commit message when chartpress pushes a new helm chart package commit to our git based helm-chart repo.